Drop gtk_init_with_args

We want to simplify our initialization code and remove all commandline
argument handling from it. The first stop for this is to reduce the
number of gtk_init variants we have.

gdk: Skip GdkVulkanContext methods when building introspection data

We cannot use Vulkan types through introspection anyway.

wayland: Push NULL buffer when hiding a GdkWindow

This is how windows are meant to be hidden as per the wayland
protocol, there's no need to destroy the xdg_surface and other
interfaces.

Also, rename gdk_wayland_window_hide_surface() to clear_surface(),
as that's what it does.

https://bugzilla.gnome.org/show_bug.cgi?id=773686

combobox: Avoid a pointless assignment

Don’t get the active item pointer before the grid/non-grid conditional,
because if we’re in grid mode, we re-get it before selecting it anyway.

combobox: Also preselect active item in grid popup

i.e. when wrap-width > 0. This was only being done for non-grid cases.
So, ComboBoxes in grid mode did not indicate their selection when popped
up and required users to keynav from ‘nothing’ (at the top-left) to the
item they wanted to select. By selecting the active item in advance, now
it’s highlighted & acts as the starting point for keynav around the grid

scale: Really fix documentation of set_digits()

This previously only mentioned its effect on the displayed value, and
even after the previous commit, its rounding of the actual value upon
change still reads like too much of an afterthought. Worse, it wasn’t
mentioned at all in the doc for the @digits parameter. Change this to
emphasise rounding always occurs and the displayed value is secondary.

scale: doc: set_digits doesn’t round retroactively

Whether it should is an open question, but for now, the documentation
should clearly indicate that currently rounding is only applied upon
changes to the value, not to the existing value when ::digits changes.
This is already clear in the doc for the underlying Range::round-digits.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

Scale: Always sync ::digits to Range::round-digits

The documents state that gtk_scale_set_digits() “causes the value of the
adjustment to be rounded off to this number of digits, so the retrieved
value matches the value the user saw.” Note the lack of any condition.

But in fact, if draw-value was false, rounding was disabled on the base
Range, so values that weren’t displayed weren’t rounded. This made the
docs wrong and made an apparently cosmetic detail alter functionality.

Fix by ensuring the number of digits set on Scale is always propagated
along to gtk_range_set_round_digits(), thus rounding to it in all cases
when the value changes, regardless of whether the value is displayed.

This doesn’t address the other idea from Bugzilla: that changing the
number of digits should clamp the _existing_ value if it’s more precise.
This contradicts digits docs in the base Range, but the above from Scale
can be read as implying it’ll happen. For now, that’s an open question.

https://bugzilla.gnome.org/show_bug.cgi?id=358970

combobox: Work around popup handler altering model

GtkFileChooserButton installs a handler for the popped-up signal, which
refilters the menu, in order to hide the “(None)” item from the popup
if it was previously selected in the ComboBox. This oddity means that:

 • Until recently, this item would be selected in the menu shell, which
   would then be popped up and change the selection away from that item.
   This was therefore redundant (more on which below!) but benign.

 • After the patch for https://bugzilla.gnome.org/show_bug.cgi?id=771242
   however, this causes a critical assertion fail, as now we stash the
   originally selected item in a pointer so that it can be selected only
   after realisation/popup – but by that stage, the model has just been
   refiltered and the previous pointer no longer refers to a valid item.

This commit works around this problem by, after popping up the menu,
getting the active item again, in case a popped-up handler has gone and
invalidated the pointer to the active item that we saved before popup.

If a handler does this, everything done to find/use the original item is
pointless. But this avoids the ugly critical in FileChooserButton, while
not harming every other ComboBox that doesn’t mess with its model while
popping up (hopefully the vast majority), and it’s very difficult to
imagine a way to check if the active item is /going to/ be hidden later)

combobox: Don’t select active item if it’s hidden

I hope no one ever actually brings such a silly item into this world,
but this achieves symmetry with the similar checks immediately after.

combobox: Move variables into narrowest scopes

Update Slovak translation

vulkan: Fix copy/paste error

vulkan: Add compiled shaders

I forgot to include them when adding the shaders.

vulkan: Compute right size for outset shadow

We were emitting the box and not the shadow box as the rectangle. So we
didn't draw any shadows. Oops.

css: Remove deprecated number-as-pixels compatibility

Previously, for compatibility with GTK 3.0, we allowed specifying
numbers without units and interpreted them as pixels, even when the CSS
specification didn't.

Remove that now that we can break API.

vulkan: Add support for unblurred box-shadow

rendernode: Add getters for box-shadow nodes

docs: Remove mention of GdkColor

Revert "filechooser portal: Make sure we can save"

This reverts commit 4875c689a0e826604953965ffdb71d8b82cae364.

This was a thinko. Writable is not actually settable from the
application side, but only for the user, from the backend side.

filechooser portal: Make sure we can save

Explicitly request files for saving to be writable, otherwise
we are at the mercy of the portal, which currently gets it wrong.

gdk/wayland: Always get the seat's key modifiers from the GdkKeymap

Elsewhere we already go through the keymap to get modifiers so we
should do the same here. In fact, this was relying on xkb modifier
mask values being bitwise compatible with GdkModifierType which isn't
necessarily true.

https://bugzilla.gnome.org/show_bug.cgi?id=770112

gdk/wayland: Don't add GDK_META_MASK to gdk mod masks if MOD1 is set

Gtk+ treats MOD1 as a synonym for Alt, and does not expect it to be
mapped around, so we should avoid adding GDK_META_MASK if MOD1 is
already included to avoid confusing gtk+ and applications that rely on
that behavior.

https://bugzilla.gnome.org/show_bug.cgi?id=770112

wayland: Handle subsurface as popup parent

When a subsurface is used as a parent of a popup, GDK needs to traverse
up to the transient-for as the next parent, to properly find the parent
used by the popup positioner. This is because the parent of a popup
must always either be an xdg_popup or an xdg_surface, but traversing
the "parent" (in GDK terms) upwards from a subsurface will end up on
the fake root window before we hit the actual parent (in Wayland terms).

https://bugzilla.gnome.org/show_bug.cgi?id=776225

vulkan: Add fallback message for node-as-texture

This is mainly to motivate me to implement it.

vulkan: Generate clip shaders from same source

Instead of having 3 different shaders for the different clipping
versions, just have one shader and use a preprocessor define to use
different clip functions.

That preprocessor define is set in the Makefile.

Also use foo.frag and foo.vert as the file extensions instead of using
foo.frag.glsl and foo.vert.glsl, as that's what glslc suggests as
extension.

vulkan: Add clip.vert.glsl

Implement clipping the same way as in the last commit for the
fragment shaders.

vulkan: Add a clip.frag.glsl include

This include is supposed to handle clipping for the different clipping
methods.

So far, we only use it in the rounded rect cases.

vulkan: Add push constants to fragment shader

That way we don't need to move the clip rounded rect manually through
the vertex shader into the fragment shader but can just look at the push
constants.

Simplifies shaders a lot.

vulkan: Move push constants into their own header

This is the first step towards easing maintenance of the Vulkan shaders
by moving common code into headers.

wayland: avoid an unnecessary g_list_length call

https://bugzilla.gnome.org/show_bug.cgi?id=777363

3.89.3

We no longer have gdktestutils

Update the docs build to reflect that.

Updated Spanish translation

Updated Spanish translation

switch: Fix build

Partial revert of 2b6b5c1c8fecdd66172516812ed3f1825619dafc, which
removed a little too much.

switch: Remove some unused code

gtkentryprivate: Remove unused prototype

scalebutton: Use GtkButon:icon-name in the ui file

spinbutton: Restore gtk3 focus behavior

Focus the spinbutton on button press, never focus the buttons.

inspector: Use GtkButton:icon-name in ui files

entry: Remove unused struct member

gtk4-section: Add GtkInfoBar:revealed setter+getter

wayland: avoid 0 width/height anchor rectangle

Passing a rectangle with zero width or height to xdg_shell-v6
set_anchor_rect() will cause a protocol error and terminate the client,
as with gedit when pressing the Win key.

Reason for this is because the rectangle used to set the anchor comes
from gtk_text_layout_get_iter_location() which uses the pango layout
width/height, which can be empty if there is not character at the given
location.

Make sure we don't use 0 as width or height as an anchor rectangle to
avoid the protocol error, and compensate the logical position of the
given rectangle if the size is changed, so that the actual position
remains as expected by the client.

https://bugzilla.gnome.org/show_bug.cgi?id=777176

Visual Studio builds: Update gsk project

Another Vulkan renderer source was added, so include them in the builds.

grid: Fix plural error in docs' @Short_description

Update Catalan translation

snapshot: Rename gtk_snapshot_translate_2d()

It's now called gtk_snapshot_offset().

Update Catalan translation

snapshot: Rename append APIs

Instead of having gtk_snapshot_append_foo_node(), just have
gtk_snapshot_append_foo(). Nobody needs to know that this internally
uses nodes.

cssimage: Fallback images have no aspect ratio

The aspect ratio for fallback image was incorrectly set to 1.

Reftest is included.

cssimage: Scale images to contain, not cover

Images with just an aspect ratio, but without a size, should be scaled
to be fully visible in the given area.
But we scaled them to completely cover the given area, which made them
partially invisible.

Reftest included.

snapshot: Redo pop() API

gtk_snapshot_pop() => removed
gtk_snapshot_pop_and_append() => gtk_snapshot_pop()

So now there is no way to get a rendernode out of the snapshotting API
until you gtk_snapshot_finish().

snapshot: Add gtk_snapshot_push_blend()

and use it for backgrounds.

reftests: Fix test

No, GtkActions don't need to be part of this test.

stack: Build stored render node with custom Snapshot object

This is in preparation for API changes in GtkSnapshot.

Add gtk_snapshot_push_cross_fade()

... and use it.

The function is a bit awkward because it requires 2 calls to
gtk_snapshot_pop(), but once you accept that, it's very convenient to
use, as can be seen by the 2 implementations.

mir: use modal window hint

widget-factory: Add a system tab to the about dialog

Just for trying it out.

about dialog: Add a "system" tab

This is a free-form tab that can contain information about the
system environment. To see it, set GtkAboutDialog::system-information
to a non-NULL value.

https://bugzilla.gnome.org/show_bug.cgi?id=776604

actionbar: Notify when :revealed changes

infobar: Fix copy&paste error in documentation

Of course you shouldn't call gtk_dialog_response on a GtkInfoBar.

infobar demo: Adapt to GtkInfoBar changes

Bind GtkToggleButton:active to GtkInfoBar:revealed instead of :visible.

infobar: Add :revealed property

infobar: Keep GParamSpecs around

So we can use them in notify_by_pspec

x11: Add call to XInitThreads()

The Mesa Vulkan drivers need XInitThreads() being called, because their
implementation has to use threads.

And I don't want to make the call depend on if Vulkan is compiled in
because that makes GTK's X11 behavior depend on compile-time flags, so
it's always called.

More filter documentation updates

Actually document the filter property, not just -gtk-icon-filter.

Correct the -gtk-icon-filter docs

I was overlooking that it is possible to specify multiple
functions. Also add a reference to the relevant spec draft.

build: List resource files explicitly

This way, we ensure that files that are built during make always get
properly listed. And we ensure that creating the resources actually
depends on them.

shortcutswindow: Make dispose work properly

tests: Add a rendernode test for borders

vulkan: Add shader for border rendering

GtkSnapshot: Always use int for the translation

We already take ints when setting the translation, so it can't
currently take any other values. Additionally, I was seeing large
costs in int -> double -> int for the rects in
gtk_snapshot_clips_rect(), as all callers really are ints (widget
allocations) and the clip region is int-based.

This change completely cleared a 2% rectangle_init_from_graphene from
the profile and is likely to have nice performance effects elsewhere
too.

widget: Avoid typechecks when accessing ->clip and ->allocation

GtkCSSImage: Avoid some type checks

The width/height/aspect getters are called a lot, and almost all
callers already verify it from _gtk_css_image_get_concrete_size (),
so just skip these checks.

Avoid some more type checks for internal calls

gtkcssgadget: Use private non-checking versions of gtk_widget calls

This avoids a lot of checking overhead.

GtkSnapshot: Reuse snapshot state objects

Rather than allocate new ones all the time we reuse the previous ones.
We just clear them and save them in the parent for later reuse.

GtkSnapshot: Move collect data to a union in the state

This means we allocate the collect data with the state, avoiding
an extra allocation. Also, a union means every state object
is the same size and we could reuse the state objects.

Snapshot: Only record names if inspector is recording

Otherwise we do a lot of allocations and vprintf calls which are
not used.

gsk_rounded_rect_init_copy: Don't normalize

This was showing up quite high on the profiles, and there is
no real reason for copy to normalize, as the source is a
GskRoundedRect which should be normalized already unless
you did something very strange (and then you should have normalized
manually).

Skip dynamic type check in css value getters

This gets called a lot during snapshotting, and this change
alone brings down snapshot time by almost 10% in a syntethic
snapshot test.

Some updates to the migration guide

Document -gtk-icon-filter

Not much detail here yet, but the syntax is documented.

cssimage: Fix cross-fade

Simgle image cross-fade opacity was computed the wrong way, which caused
weird fade-in/out animations, for example in flat buttons.

I messed this up when porting cross-fades to snapshot().

Remove an unnecessary check

Since the demise of theme engines, we can no longer hit
the case of id >= GTK_CSS_PROPERTY_N_PROPERTIES. So don't
check for this in a very frequently called function.

Use an internal parameter check here as well

We should not slow down the core parts of the css machinery
with type checks.

Use gtk_internal_return_val_if_fail here

It seems odd to have checks in just a few functions,
so switch everything over to use the internal versions.

Use _gtk_widget_get_window more

This avoids type checks in places where we know it is safe.

Refactor some css transition code slightly

Reshuffling things a bit to avoid a bunch of NULL and
type checks.

Use the .symbolic.png assets in Adwaita

Using an image() fallback from svg to png doesn't make too
much sense, since the svg is always used (unless librsvg is
not present), while the png icon is faster and cheaper to
load and thus preferable.

gdk/wayland: Handle non-existant gsettings keys

Since we're a library, crashing on gsettings keys, whose presence is out
of our control, isn't appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=775846

gdk/wayland: Add support for the gtk-enable-primary-paste gsetting

The gsetting was recently added so that we can have this configurable
on the wayland backend too.

https://bugzilla.gnome.org/show_bug.cgi?id=775846

mir: fix compile-time warnings

revealer: Fix a typo in a function doc

Also, "ie" wasn't very clear, but fixing that to "i.e." would cause
truncation of the summary when processed by bindings using doxygen. So,
I replaced it with "in other words", which is no _less_ clear, at least.

mir: properly handle empty clipboard

https://bugzilla.gnome.org/show_bug.cgi?id=775732

Visual Studio builds: Move project files to win32/

It was suggested that the project files to be moved to win32/, so that we can
have one less layer of directories we need to go down into to reach the project files.

build/Makefile.msvcproj: Improve documentation

Tell people about what happens when generating projects when Visual
Studio 2013 or later is required, and mention that the .headers are
only needed when headers need to be copied.

cssgadget: Compute clip correctly

We were computing it relative to the gadget allocation, but it should be
relative to the widget allocation.

widget: Document child/sibling accessors